-
-
Notifications
You must be signed in to change notification settings - Fork 0
com.utilities.websockets 1.1.0 #18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- fix webgl build regressions when using multi-threaded environments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the com.utilities.websockets package to version 1.0.7, addressing WebGL function name collisions and updating test infrastructure. The changes ensure the WebSocket plugin works correctly in multi-threaded WebGL environments by renaming internal functions to avoid conflicts with other packages.
- Fixed WebGL WebSocket.jspre function name collisions by renaming
initDynCalls
toinitDynCalls_websockets
- Updated test server URI from
wss://echo.websocket.events
towss://echo.websocket.org
- Updated Unity editor version and package dependencies
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
ProjectSettings/ProjectVersion.txt | Updated Unity editor version from 2021.3.45f1 to 2021.3.45f2 |
Packages/manifest.json | Updated dependency versions for IDE packages and build pipeline |
Packages/com.utilities.websockets/package.json | Bumped package version from 1.0.6 to 1.0.7 |
Packages/com.utilities.websockets/Tests/TestFixture_01.cs | Updated test to use new echo server address and introduced constant for server URI |
Packages/com.utilities.websockets/Samples~/WebsocketDemo/WebSocketDemo.unity | Updated demo scene to use new echo server address |
Packages/com.utilities.websockets/Runtime/Plugins/WebSocket.jspre | Renamed initDynCalls to initDynCalls_websockets to prevent name collisions |
.gitignore | Added boot.config to ignored files |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Utilities.Websockets/Packages/com.utilities.websockets/Runtime/Plugins/WebSocket.jspre
Show resolved
Hide resolved
- fixed webgl WebSocket.jspre function name collisions - update testing server uri - update unit tests - update validation workflows - update unity editor to 2021.3.45f2 (88f88f591b2e)
Uh oh!
There was an error while loading. Please reload this page.